Skip to content

feat: Update Seedream 4.5 resolution limits to 4704x4704 #11988

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
sasankkurnella wants to merge 1 commit into
base: master
Choose a base branch
from
Open

feat: Update Seedream 4.5 resolution limits to 4704x4704 #11988

sasankkurnella wants to merge 1 commit into from
+7 −4

Conversation

@sasankkurnella
Copy link

@sasankkurnella sasankkurnella commented Jan 20, 2026 •
edited by github-actions bot
Loading

Fixes #11394

The Seedream 4.5 model supports up to 4704×4704 resolution, but the node was
limiting it to 4096×4096.

Changes

  • Updated UI schema max values for width/height to 4704
  • Added model-specific validation so Seedream 4.0 remains at 4096 max
  • Seedream 4.5 now correctly allows up to 4704×4704

Backward Compatibility

  • Seedream 4.0 behavior is unchanged (still limited to 4096)
  • Only relaxes constraints for Seedream 4.5 to match model capability

API Node PR Checklist

Scope

  • Is API Node Change

Pricing & Billing

  • Need pricing update
  • No pricing update

If Need pricing update:

  • Metronome rate cards updated
  • Auto‑billing tests updated and passing

QA

  • QA done
  • QA not required

Comms

  • Informed Kosinkadink

feat: Update Seedream 4.5 resolution limits to 4704x4704
0b5a472 
Fixes Comfy-Org#11394

The Seedream 4.5 model supports up to 4704x4704 resolution, but the node
was limiting it to 4096x4096. This commit:

- Updates the UI schema max values for width/height to 4704
- Adds model-specific validation so Seedream 4.0 remains at 4096 max
- Seedream 4.5 now correctly allows up to 4704x4704

This is backward compatible - Seedream 4.0 behavior is unchanged.
@bigcat88 bigcat88 self-assigned this Jan 20, 2026
@bigcat88
Copy link
Contributor

bigcat88 commented Jan 21, 2026

I noticed that this range exceeds what is listed in the documentation.

Is there a secondary source or a changelog indicating this is officially supported? If not, I'm concerned that BytePlus might block these requests in the future.

@comfy-pr-bot
Copy link
Member

comfy-pr-bot commented Jan 22, 2026

Test Evidence Check

⚠️ Warning: Test Explanation Missing

If this PR modifies behavior that requires testing, a test explanation is required. PRs lacking applicable test explanations may not be reviewed until added. Please add test explanations to ensure code quality and prevent regressions.

⚠️ Warning: Visual Documentation Missing

If this PR changes user-facing behavior, visual proof (screen recording or screenshot) is required. PRs without applicable visual documentation may not be reviewed until provided.

You can add it by:

  • GitHub: Drag & drop media directly into the PR description
  • YouTube: Include a link to a short demo

@sasankkurnella
Copy link
Author

sasankkurnella commented Jan 22, 2026

Update: Documentation Research

After thorough research, I was unable to find official BytePlus/ByteDance documentation confirming 4704×4704 support for Seedream 4.5. The available documentation consistently states a maximum of 4096×4096.

Sources checked:

  • BytePlus official docs
  • Third-party API providers (fal.ai, Replicate, Segmind, Wavespeed)

The original issue #11394 claims the model supports 4704×4704, but without official documentation or API test results, I cannot independently verify this.

Regarding the bot warnings:
This is a backend validation change with no visual UI modifications. Testing would require making an API call with the higher resolution values.

Questions for maintainers:

  1. Does anyone have evidence (successful API response) confirming 4704×4704 works with Seedream 4.5?
  2. Should we keep this PR open pending verification, or close it until official documentation confirms the higher limit?

I'm happy to adjust or close the PR based on your guidance. I don't currently have API credits to test this myself.

@bigcat88
Copy link
Contributor

bigcat88 commented Jan 22, 2026

2. Should we keep this PR open pending verification, or close it until official documentation confirms the higher limit?

I'm happy to adjust or close the PR based on your guidance. I don't currently have API credits to test this myself.

I can propose to close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@comfyanonymous comfyanonymous Awaiting requested review from comfyanonymous comfyanonymous is a code owner

@Kosinkadink Kosinkadink Awaiting requested review from Kosinkadink Kosinkadink is a code owner

@guill guill Awaiting requested review from guill guill is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@bigcat88 bigcat88

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Seedream 4.5 resolution limits: update max dimension and max pixel count to 4704×4704 (from 4096×4096)

3 participants

@sasankkurnella @bigcat88 @comfy-pr-bot